Skip to content

feat(ClipboardCopy): added truncation for inlinecompact variant#11610

Merged
wise-king-sullyman merged 8 commits into
patternfly:mainfrom
thatblindgeye:iss10890_clipboardTruncate
Mar 12, 2025
Merged

feat(ClipboardCopy): added truncation for inlinecompact variant#11610
wise-king-sullyman merged 8 commits into
patternfly:mainfrom
thatblindgeye:iss10890_clipboardTruncate

Conversation

@thatblindgeye

Copy link
Copy Markdown
Contributor

What: Closes #10890

Additional issues:

@thatblindgeye thatblindgeye requested review from a team, christianvogt, gitdallas, mfrances17 and tlabaj and removed request for a team March 5, 2025 19:42
@patternfly-build

patternfly-build commented Mar 5, 2025

Copy link
Copy Markdown
Collaborator

@gitdallas

Copy link
Copy Markdown
Contributor

played around a little with the truncate props there
image

@gitdallas gitdallas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actually, i wonder if we can make sure that the icon is inline with the text? seems like it is always going to the next level:
image

@thatblindgeye thatblindgeye force-pushed the iss10890_clipboardTruncate branch from 7e8e6d3 to db7d624 Compare March 11, 2025 20:07
@thatblindgeye thatblindgeye force-pushed the iss10890_clipboardTruncate branch from db7d624 to 96a3823 Compare March 12, 2025 15:54

@gitdallas gitdallas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm now, thanks 🚀

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ClipboardCopy - Allow truncate flexibility

6 participants